Skip to content

Comments

Show native app only elements on browser App renders in dev mode. #15330

Open
vlbee wants to merge 5 commits intomainfrom
vb/app-default-l2a-devmode
Open

Show native app only elements on browser App renders in dev mode. #15330
vlbee wants to merge 5 commits intomainfrom
vb/app-default-l2a-devmode

Conversation

@vlbee
Copy link
Member

@vlbee vlbee commented Feb 10, 2026

What does this change?

The Follow button and Listen to Article button on articles require a the native bridget service to render, so they do not appear on App rendered articles viewed in a web browser for local development: eg. http://localhost:3030/AppsArticle/...

This defaults the components to show up in localhost dev mode to facilitate development.

Why?

Screenshots

image

@vlbee vlbee requested a review from a team as a code owner February 10, 2026 15:44
@github-actions
Copy link

Hello 👋! When you're ready to run Chromatic, please apply the run_chromatic label to this PR.

You will need to reapply the label each time you want to run Chromatic.

Click here to see the Chromatic project.

@vlbee vlbee added run_chromatic Runs chromatic when label is applied fix Departmental tracking: fix labels Feb 10, 2026
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Feb 10, 2026
Copy link
Member

@arelra arelra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this.

On balance I think we would prefer if components render deterministically with their expected dependencies (e.g. Bridget) as they would in PROD. This makes rendering and testing deterministic without introducing special (perhaps surprising) workarounds for DEV.

That would mean running DCAR locally with a simulator for the full e2e experience.

We do appreciate this is a slower workflow and we have a long term goal of implementing dependency injection in DCAR that would allow a version of Bridget for local testing purposes.

fyi @JamieB-gu

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Departmental tracking: fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants